Skip to content

[ci] : add First-Time Contributor#341

Open
7h3-3mp7y-m4n wants to merge 1 commit intoscore-spec:mainfrom
7h3-3mp7y-m4n:cicd
Open

[ci] : add First-Time Contributor#341
7h3-3mp7y-m4n wants to merge 1 commit intoscore-spec:mainfrom
7h3-3mp7y-m4n:cicd

Conversation

@7h3-3mp7y-m4n
Copy link
Copy Markdown
Contributor

@7h3-3mp7y-m4n 7h3-3mp7y-m4n commented Aug 17, 2025

Description

This PR adds a new workflow for new contributors.
fixes #340

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • New chore (expected functionality to be implemented)

Checklist:

  • My change requires a change to the documentation.
    • I have updated the documentation accordingly.
  • I've signed off with an email address that matches the commit author.

@7h3-3mp7y-m4n
Copy link
Copy Markdown
Contributor Author

7h3-3mp7y-m4n commented Aug 17, 2025 •
edited
Loading

We can improve the comments! (I'd love to see suggestions/also if that feature is needed).
Also, it needed some changes

mevrin-ueat
mevrin-ueat reviewed Aug 19, 2025
View reviewed changes
.github/workflows/first-time-contribution.yaml
labels: ['first-time contributor']
});

// Check DCO
Copy link
Copy Markdown

@mevrin-ueat mevrin-ueat Aug 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am pretty sure DCO is already covered

@mathieu-benoit
Copy link
Copy Markdown
Contributor

mathieu-benoit commented Sep 3, 2025 •
edited
Loading

Thanks, @7h3-3mp7y-m4n, this is a super great idea, love it!

Before moving forward with this one, let's make sure we finalize this one first: #330.

DCO is definitely something new from new contributors and has been generated some frictions... the "how to" is discovered in CI that is failing... not a good experience, so I think I like the idea to document here too.

Also, for reference and inspiration, let's check what we could reuse from the Microcks project: https://github.com/microcks/microcks/blob/master/.github/workflows/welcome-new-users.yml.

@7h3-3mp7y-m4n
Copy link
Copy Markdown
Contributor Author

7h3-3mp7y-m4n commented Sep 3, 2025

Thanks, @7h3-3mp7y-m4n, this is a super great idea, love it!

Before moving forward with this one, let's make sure we finalize this one first: #330.

DCO is definitely something new from new contributors and has been generated some frictions... the "how to" is discovered in CI that is failing... not a good experience, so I think I like the idea to document here too.

Also, for reference and inspiration, let's check what we could reuse from the Microcks project: https://github.com/microcks/microcks/blob/master/.github/workflows/welcome-new-users.yml.

For sure, I'll look at it!

@astromechza astromechza self-requested a review September 4, 2025 13:05
@7h3-3mp7y-m4n @astromechza
[ci] : add First-Time Contributor
d03c3d4 
Signed-off-by: 7h3-3mp7y-m4n <emailtorash@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@astromechza astromechza Awaiting requested review from astromechza

1 more reviewer

@mevrin-ueat mevrin-ueat mevrin-ueat left review comments

Reviewers whose approvals may not affect merge requirements

At least 0 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[feature request] [ci] Add a label and a comment for first-time contributors

3 participants

@7h3-3mp7y-m4n @mathieu-benoit @mevrin-ueat